-
Notifications
You must be signed in to change notification settings - Fork 197
fix(docs): remove broken @std/log link from README #874
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
The jsr.io/@std/log URL returns 404. Removed the broken link while keeping all other working logger examples (pino, winston, bunyan, consola, signale). 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]>
felixfbecker
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The link works for me? https://jsr.io/@std/log
|
Hi @felixfbecker! Thanks for reviewing. I just re-checked and The package might have been moved or renamed. The correct URL appears to be Would you like me to:
Happy to adjust based on your preference! |
|
A more compelling reason to remove or adjust it can be found in the package's issue #6124 thread, which is linked in the url's page at "@std/log is likely to be removed in the future. Consider using Open Telemetry."
|
|
It appears that it returns a different HTTP response when you use |
|
@dtmeadows wdyt about |
|
Some people are still relying on it denoland/std#6124 (comment), what's the benefit in removing the link? |
|
Thank you for the feedback, @RobertCraigie. You raise a fair point - if people are still relying on it and the link works in browser, there's no strong reason to remove it. I'm happy to close this PR. Appreciate you and @felixfbecker taking the time to review. |
|
Actually, to clarify - the link does return a 404 when accessed programmatically: It only works in browser due to JavaScript. So documentation links that require JS to resolve aren't ideal. Combined with @0xrdan's point that @std/log is likely to be removed per denoland/std#6124, there may still be value in updating this. That said, I understand if you'd prefer to keep it as-is. Happy either way - just wanted to clarify the original reasoning. Thank you again for reviewing. |

Summary
The
jsr.io/@std/logURL in the README returns 404. This PR removes the broken link while preserving all other working logger examples (pino, winston, bunyan, consola, signale).Test plan
jsr.io/@std/logreturns 404🤖 Generated with Claude Code